Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(rosco): Revert Make pause_before configurable (#224) #226

Merged
merged 1 commit into from
Nov 3, 2017
Merged

fix(rosco): Revert Make pause_before configurable (#224) #226

merged 1 commit into from
Nov 3, 2017

Conversation

tpulmano
Copy link
Contributor

@tpulmano tpulmano commented Nov 2, 2017

This reverts commit 8f05647.

Per this Packer issue pause_before isn't being expanded: hashicorp/packer#5115

@avram
Copy link
Contributor

avram commented Nov 3, 2017

This was my mistake. We identified that we wanted this configurable and I eagerly PR'ed the obvious "quick fix" -- but Packer doesn't let us template that value.

@duftler
Copy link

duftler commented Nov 3, 2017

LGTM.
I suppose you could add an additional shell provisioner just before and have it's referenced script do nothing but pause for some parameterized amount of time.

@duftler duftler merged commit 9a84499 into spinnaker:master Nov 3, 2017
opsmxdemo pushed a commit to OpsMx/rosco that referenced this pull request Sep 15, 2020
opsmxdemo pushed a commit to OpsMx/rosco that referenced this pull request Sep 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants